Add Projects api and rework the tests#384
Add Projects api and rework the tests#384klarkc wants to merge 34 commits intogithub-tools:masterfrom
Conversation
This reverts commit 54db6f3.
clayreimann
left a comment
clayreimann
left a comment
There was a problem hiding this comment.
Looks good, there are still 2 lint errors that it would be great if you fixed or forcibly ignored. (I know you didn't introduce one of them)
|
I discovered that the number of testing repos was causing a lot of trouble in tests with timeouts, so I wrote the clearRepo function that basically clear all test repos from the test user. But because of this change, some other tests that do not create testing repos, now are broken. I am working on these tests to first create the testing repo and after that do the tests. |
klarkc
changed the title
|
I had to do the same thing with teams, so I wrote also the |
|
@clayreimann The travis is testing with concurrency 1? As the tests are made in only one user and organization, the running of parallel tests can cause some unknown errors. Just checking. The tests are running ok in local machine btw 😭. |
2 participants
Adds the new Projects API
The tests are passing, but the lint contains errors from master.
The lib folder are not used in the NPM package, so I put it in the .npmignore